Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[cherry-pick] Fix: GRPC healthcheck should not have keepalive time #5921

Conversation

nginx-bot
Copy link
Contributor

Proposed changes

#4996

  • Ensure keepalive is not added to grpc health check
  • Improve formatting of grpc health check directive
  • Remove flaky label from grpc related tests (as they are running locally fine, open to adding this back however)
  • Add test for grpc health check by simulating a health check fail, where the status does not match the expected status, and expecting a line in the nginx log
  • Update snapshots

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

* fix template file, and fix test

* add test for GPRC health check fail message, remove flaky label from prior tests, clean up health check template

* update snap

* update snapshots

* remove newline

* fix tests
@nginx-bot nginx-bot requested a review from a team as a code owner July 2, 2024 13:57
@nginx-bot nginx-bot added bug An issue reporting a potential bug dependencies Pull requests that update a dependency file go Pull requests that update Go code needs cherry pick Cherry pick this PR into a release branch python Pull requests that update Python code tests Pull requests that update tests labels Jul 2, 2024
@github-actions github-actions bot removed bug An issue reporting a potential bug dependencies Pull requests that update a dependency file labels Jul 2, 2024
@j1m-ryan j1m-ryan merged commit 459d4d3 into release-3.6 Jul 2, 2024
65 checks passed
@j1m-ryan j1m-ryan deleted the cherry-pick-release-3.6-f154d3079c8858fad43397eb77e1f24c726e489d branch July 2, 2024 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go Pull requests that update Go code needs cherry pick Cherry pick this PR into a release branch python Pull requests that update Python code tests Pull requests that update tests
Projects
Status: Done 🚀
Development

Successfully merging this pull request may close these issues.

3 participants